Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(linter): Added pyproject-flake8 support #444

Merged
merged 3 commits into from
Nov 9, 2023
Merged

feat(linter): Added pyproject-flake8 support #444

merged 3 commits into from
Nov 9, 2023

Conversation

toirl
Copy link
Contributor

@toirl toirl commented Nov 3, 2023

Hi,

I love flake8, and I even more I love configuring it using pyroject.toml files, which isn't supported by flake8. However there is a package (https://github.com/csachs/pyproject-flake8) which makes this possible.

I currently run nvim-lint with a custom configuration for this linter which works well. So I thought it might be a nice idea to add this to upstream also.

Copy link
Owner

@mfussenegger mfussenegger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Could you please revert the unrelated style changes?

Is there any reason to use this over https://docs.astral.sh/ruff/ - which is faster and also supports pyproject.toml?

@toirl
Copy link
Contributor Author

toirl commented Nov 8, 2023

Hi, just reverted the changes on style.

Regarding the question for reasons against ruff: No reasons against ruff of course. I just did not used it yet. The often mentioned faster speed of ruff wasn't valuable enough for me to change my current setup. Never had issues with the speed of the other linters.

@mfussenegger mfussenegger merged commit bd9bcc0 into mfussenegger:master Nov 9, 2023
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants